MIgrating Azure pipelines to GHA workflows by lamkina · Pull Request #30 · scritical/.github

lamkina · 2026-02-03T22:50:46Z

Purpose

This PR migrates our CI infrastructure from Azure pipelines to Github Actions workflows. This will target the SR runners instead of the runners provided by Azure.

This PR will remove the Azure pipelines templates.

Expected time until merged

1-2 days

Type of change

Bugfix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (non-backwards-compatible fix or feature)
Code style update (formatting, renaming)
Refactoring (no functional changes, no API changes)
Documentation update
Maintenance update
Other (please describe)

… Docker

Copilot

Pull request overview

This PR migrates CI infrastructure from Azure Pipelines to GitHub Actions workflows. The migration removes all Azure-specific templates and configurations while introducing equivalent GitHub Actions reusable workflows. A new combine-config.py script has been added at the repository root to support configuration merging for tools like pylint and isort.

Changes:

Removed all Azure Pipeline templates and configuration files from the azure/ directory
Added new GitHub Actions reusable workflows for build, test, formatting, and linting
Created comprehensive documentation for GitHub Actions workflows in .github/workflows/README.md
Updated existing workflows to use consistent Python 3.11 and improved configuration management

Reviewed changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
`ruff.toml`	Reformatted select array and added McCabe complexity configuration
`combine-config.py`	New script for merging INI-style configuration files with improved error handling
`azure/fprettify.sh`	Removed Azure-specific fprettify script
`azure/combine-config.py`	Removed Azure-specific config merge script
`azure/clang-format.sh`	Removed Azure-specific clang-format script
`azure/azure_template.yaml`	Removed Azure pipeline template
`azure/azure_tapenade.yaml`	Removed Azure Tapenade job template
`azure/azure_style.yaml`	Removed Azure style check job template
`azure/azure_pypi.yaml`	Removed Azure PyPI deployment template
`azure/azure_build.yaml`	Removed Azure build and test job template
`azure/README.md`	Removed Azure pipelines documentation
`.github/workflows/tapenade.yaml`	New GHA workflow for Tapenade checks
`.github/workflows/pypi.yaml`	Updated Python version to 3.11
`.github/workflows/pylint.yaml`	Enhanced with config merging and Python 3.11
`.github/workflows/mypy.yaml`	Updated to use only Python 3.11
`.github/workflows/isort.yaml`	Enhanced with config merging and Python 3.11
`.github/workflows/fprettify.yaml`	New GHA workflow for Fortran formatting
`.github/workflows/format-and-lint.yaml`	Enhanced with McCabe complexity option and Python 3.11
`.github/workflows/clang_format.yaml`	New GHA workflow for C/C++ formatting
`.github/workflows/build.yaml`	New GHA workflow for Docker-based build and test
`.github/workflows/README.md`	Comprehensive documentation for GHA workflows

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

combine-config.py

.github/workflows/tapenade.yaml

.github/workflows/pylint.yaml

.github/workflows/isort.yaml

.github/workflows/fprettify.yaml

.github/workflows/clang_format.yaml

.github/workflows/pylint.yaml

.github/workflows/pypi.yaml

.github/workflows/isort.yaml

.github/workflows/format-and-lint.yaml

.github/workflows/isort.yaml

.github/workflows/README.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

lamkina · 2026-02-05T04:28:47Z

Take a look at this splinetoolbox PR to see the testing for all of the workflow updates being made.

.github/workflows/README.md

.github/workflows/build.yaml

.github/workflows/clang_format.yaml

.github/workflows/build.yaml

.github/workflows/isort.yaml

.github/workflows/tapenade.yaml

…g step to ruff workflow

… workflow to run in docker images

lamkina · 2026-02-11T18:56:37Z

@sseraj ready for review round 2 the electric boogaloo

Copilot

Pull request overview

Copilot reviewed 26 out of 29 changed files in this pull request and generated 6 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/workflows/clang_format.yaml

.github/workflows/ruff.yaml

combine-config.py

.github/workflows/README.md

README.md

.github/workflows/build.yaml

.github/workflows/README.md

.github/workflows/build.yaml

azure/fprettify.sh

.github/workflows/README.md

.github/workflows/build.yaml

.github/workflows/README.md

eirikurj · 2026-02-12T22:24:41Z

@lamkina to save time I think I will wait until @sseraj comments are addressed and do another round of reviews then

lamkina · 2026-02-14T00:35:54Z

Okay, I split out the mypy job from build.yaml into a separate mypy.yaml workflow. I also added back the prettify.sh script. Since we are removing the azure pipelines, the fprettify.sh script is not in a different location on main then where it needs to be for the workflows.

After a long battle, I decided to use the script code in the fprettify.yaml workflow until this PR is merged. Then we can change the workflow to use the script in a subsequent PR to fix the code duplication.

I also updated docs per @sseraj review. I think it's ready for a final round...fingers crossed.

lamkina and others added 2 commits January 19, 2026 22:35

Migrating from azure pipelines to GHA with minor updates for Ruff and…

9cf7a7a

… Docker

Fix strategy matrix for mypy workflow

1d0a80e

lamkina requested review from Copilot, eirikurj and sseraj February 3, 2026 22:50

Copilot AI reviewed Feb 3, 2026

View reviewed changes

sseraj reviewed Feb 4, 2026

View reviewed changes

lamkina and others added 18 commits February 4, 2026 10:50

Update .github/workflows/fprettify.yaml

b3cfd71

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update .github/workflows/clang_format.yaml

8db7f40

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Removing reference to azure in root README

88c8ce4

Removing pypi workflow

ec760cc

Adding specific ruff workflow and removing old format and lint workflow

88e9fd6

updating ruff yaml extension

ff3a1e4

Fixing indentation

34c5431

Updating ruff workflow to fix gha error

e484d0c

Testing ruff pwd

f280eb3

Fixing ruff directory issues

042b4eb

Debugging ruff formatting path

d98f177

Trying to fix path issues with gha and ruff

ea96700

Fixing an argument bug

5ac78f5

Trying to fix build bug with paths

3f51459

Making the branch name check a callable workflow

b6f5096

Fixing bug with branch workflow

342d9dc

Removing pylint

c08da3d

Improving docstrings

25b908f

lamkina requested a review from sseraj February 5, 2026 04:27

eirikurj reviewed Feb 5, 2026

View reviewed changes

lamkina added 2 commits February 10, 2026 21:52

Changing all .yml to .yaml for consistency

5baa589

Adding docker login action instead of command

e7fb61b

lamkina added 3 commits February 11, 2026 18:34

Removing MDO Lab codeowners

3f327d9

Removing import sorting by default from Ruff config and adding sortin…

2ac4151

…g step to ruff workflow

Updating readme, refactoring build workflow, and adding mypy to build…

05cfcee

… workflow to run in docker images

lamkina requested review from Copilot and eirikurj February 11, 2026 18:55

Copilot AI reviewed Feb 11, 2026

View reviewed changes

lamkina added 10 commits February 11, 2026 22:05

Minor update to the build workflow for turning intel/gcc on or off

582951c

Fixing env bug

7d7ef20

Improving the logic checks for the build workflow for intel/gcc

30a1c97

Fixing mypy

a2db2c1

Creating composite actions for build workflow to split jobs

08d7ff2

Fixing .yml to .yaml

930109c

Adding a checkout step for composite actions workflow

502dadf

Removing composite actions and trying yaml anchor approach

8d0b709

Fixing yaml anchor

0a22772

Fixing yaml anchors pt 2

8b24add

sseraj reviewed Feb 12, 2026

View reviewed changes

lamkina added 8 commits February 13, 2026 00:46

Addressing new round of comments pt 1

4adc7d8

Restoring fprettify shell script

b6571d6

Fixing secrets docs in readme

8217298

Updating workflow options section

97e5837

Readme update and accounting for local/global ruff toml configs

fd0814e

Fixing ruff config again

3286a73

Adding a standalone mypy workflow

f328a25

Updating readme with mypy workflow

ddbaa7d

lamkina force-pushed the feature-migrate_azure2gha branch from 2434991 to ddbaa7d Compare February 14, 2026 00:30

lamkina requested a review from sseraj February 14, 2026 00:35

Conversation

lamkina commented Feb 3, 2026

Purpose

Expected time until merged

Type of change

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lamkina commented Feb 5, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lamkina commented Feb 11, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

eirikurj commented Feb 12, 2026

Uh oh!

lamkina commented Feb 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants